CI test improvements #726
CI test improvements #726
Conversation
Godzilla-oss
commented
Jan 22, 2026
Godzilla-oss
commented
- cache test builds
- retry once if a test fails
- replace fail-fast with a maximum of 5 test failures
- test timeout after 1h 30m
- parallel test execution
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #726 +/- ##
===========================================
+ Coverage 68.87% 80.78% +11.91%
===========================================
Files 35 49 +14
Lines 4932 14419 +9487
===========================================
+ Hits 3397 11649 +8252
- Misses 1535 2770 +1235 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
.github/workflows/test.yaml
Outdated
| cargo llvm-cov nextest --lib --bins --tests --fail-fast --features integration-test --no-capture | ||
| cargo llvm-cov report --cobertura --output-path ./coverage/reports/cobertura.xml | ||
| cargo llvm-cov nextest --lib --bins --tests --retries 1 --max-fail 5:immediate --partition hash:${{ matrix.partition }}/2 --features integration-test --no-capture |
There was a problem hiding this comment.
will this run all the tests paralley? or create 2 parallel run streams based on the partitions?
There was a problem hiding this comment.
create 2 parallel runners
There was a problem hiding this comment.
In this line hash:${{ matrix.partition }}/2 the partition number will always be 1? Seems like an integer ops, but I might be reading it wrong.
There was a problem hiding this comment.
In this line
hash:${{ matrix.partition }}/2the partition number will always be 1? Seems like an integer ops, but I might be reading it wrong.
This is not performing a mathematical operation; the argument for --partition should be passed using this syntax.
There was a problem hiding this comment.
can you create 5 partitions?
yes we can.
There was a problem hiding this comment.
can you create 5 partitions?
yes we can.
then let's do it
Godzilla-oss
force-pushed
the
ci-imp
branch
2 times, most recently
from
7f294fd to
dc215f0
Compare
- cache test builds - retry once if a test fails - replace fail-fast with a maximum of 3 test failures - test timeout after 1h 30m - parallel test execution
3 participants
